gdk: Remove gdk_event_put(), peek() etc
authorBenjamin Otte <otte@redhat.com>
Tue, 12 Dec 2017 23:35:16 +0000 (00:35 +0100)
committerBenjamin Otte <otte@redhat.com>
Tue, 12 Dec 2017 23:56:52 +0000 (00:56 +0100)
We don't want to treat events like they don't belong to displays. So
instead, people should use gdk_display_put/peek/get_event().

docs/reference/gdk/gdk4-sections.txt
gdk/gdkevents.c
gdk/gdkevents.h
gdk/wayland/gdkdnd-wayland.c
gdk/wayland/gdkselection-wayland.c
gdk/x11/gdkdevicemanager-core-x11.c
gdk/x11/gdkdnd-x11.c
gdk/x11/gdkeventsource.c

index a22b4d36fb8ee5f3e447409d0970f0670a4c75d8..e18a2d5abfcc19d7d2627cff02e890b020f25ed7 100644 (file)
@@ -701,10 +701,6 @@ GDK_BUTTON_MIDDLE
 GDK_BUTTON_SECONDARY
 
 <SUBSECTION>
-gdk_events_pending
-gdk_event_peek
-gdk_event_get
-gdk_event_put
 gdk_event_new
 gdk_event_copy
 gdk_event_free
index 0cafdb27fe294c5c27b3679fadacd29b593540d1..054ca11206cd46a6a115c8534f59d11529c89a05 100644 (file)
@@ -352,135 +352,6 @@ gdk_event_handler_set (GdkEventFunc   func,
   _gdk_event_notify = notify;
 }
 
-/**
- * gdk_events_pending:
- *
- * Checks if any events are ready to be processed for any display.
- *
- * Returns: %TRUE if any events are pending.
- */
-gboolean
-gdk_events_pending (void)
-{
-  GSList *list, *l;
-  gboolean pending;
-
-  pending = FALSE;
-  list = gdk_display_manager_list_displays (gdk_display_manager_get ());
-  for (l = list; l; l = l->next)
-    {
-      if (_gdk_event_queue_find_first (l->data))
-        {
-          pending = TRUE;
-          goto out;
-        }
-    }
-
-  for (l = list; l; l = l->next)
-    {
-      if (gdk_display_has_pending (l->data))
-        {
-          pending = TRUE;
-          goto out;
-        }
-    }
-
- out:
-  g_slist_free (list);
-
-  return pending;
-}
-
-/**
- * gdk_event_get:
- * 
- * Checks all open displays for a #GdkEvent to process,to be processed
- * on, fetching events from the windowing system if necessary.
- * See gdk_display_get_event().
- * 
- * Returns: (nullable): the next #GdkEvent to be processed, or %NULL
- * if no events are pending. The returned #GdkEvent should be freed
- * with gdk_event_free().
- **/
-GdkEvent*
-gdk_event_get (void)
-{
-  GSList *list, *l;
-  GdkEvent *event;
-
-  event = NULL;
-  list = gdk_display_manager_list_displays (gdk_display_manager_get ());
-  for (l = list; l; l = l->next)
-    {
-      event = gdk_display_get_event (l->data);
-      if (event)
-        break;
-    }
-
-  g_slist_free (list);
-
-  return event;
-}
-
-/**
- * gdk_event_peek:
- *
- * If there is an event waiting in the event queue of some open
- * display, returns a copy of it. See gdk_display_peek_event().
- * 
- * Returns: (nullable): a copy of the first #GdkEvent on some event
- * queue, or %NULL if no events are in any queues. The returned
- * #GdkEvent should be freed with gdk_event_free().
- **/
-GdkEvent*
-gdk_event_peek (void)
-{
-  GSList *list, *l;
-  GdkEvent *event;
-
-  event = NULL;
-  list = gdk_display_manager_list_displays (gdk_display_manager_get ());
-  for (l = list; l; l = l->next)
-    {
-      event = gdk_display_peek_event (l->data);
-      if (event)
-        break;
-    }
-
-  g_slist_free (list);
-
-  return event;
-}
-
-static GdkDisplay *
-event_get_display (const GdkEvent *event)
-{
-  if (event->any.window)
-    return gdk_window_get_display (event->any.window);
-  else
-    return gdk_display_get_default ();
-}
-
-/**
- * gdk_event_put:
- * @event: a #GdkEvent.
- *
- * Appends a copy of the given event onto the front of the event
- * queue for event->any.window’s display, or the default event
- * queue if event->any.window is %NULL. See gdk_display_put_event().
- **/
-void
-gdk_event_put (const GdkEvent *event)
-{
-  GdkDisplay *display;
-  
-  g_return_if_fail (event != NULL);
-
-  display = event_get_display (event);
-
-  gdk_display_put_event (display, event);
-}
-
 static GHashTable *event_hash = NULL;
 
 /**
@@ -729,7 +600,7 @@ gdk_event_copy (const GdkEvent *event)
     }
 
   if (gdk_event_is_allocated (event))
-    _gdk_display_event_data_copy (event_get_display (event), event, new_event);
+    _gdk_display_event_data_copy (gdk_event_get_display (event), event, new_event);
 
   return new_event;
 }
@@ -815,7 +686,7 @@ gdk_event_free (GdkEvent *event)
       break;
     }
 
-  display = event_get_display (event);
+  display = gdk_event_get_display (event);
   if (display)
     _gdk_display_event_data_free (display, event);
 
index 9318e8854b72b18e49915a4746e65a421abb7e0a..e9e6f030f8bfbeee94bbb19faa3240a15ca94540 100644 (file)
@@ -518,16 +518,6 @@ GType     gdk_event_get_type            (void) G_GNUC_CONST;
 GDK_AVAILABLE_IN_3_14
 GType     gdk_event_sequence_get_type   (void) G_GNUC_CONST;
 
-GDK_AVAILABLE_IN_ALL
-gboolean  gdk_events_pending           (void);
-GDK_AVAILABLE_IN_ALL
-GdkEvent* gdk_event_get                        (void);
-
-GDK_AVAILABLE_IN_ALL
-GdkEvent* gdk_event_peek                (void);
-GDK_AVAILABLE_IN_ALL
-void      gdk_event_put                        (const GdkEvent *event);
-
 GDK_AVAILABLE_IN_ALL
 GdkEvent* gdk_event_new                 (GdkEventType    type);
 GDK_AVAILABLE_IN_ALL
index 2a2c56c51f310f2afbd7b38592df1c0daf19c8e3..208b71e20ddc912254b8aa243d16b31653988114 100644 (file)
@@ -137,7 +137,7 @@ _gdk_wayland_drag_context_emit_event (GdkDragContext *context,
   event->dnd.y_root = GDK_WAYLAND_DRAG_CONTEXT (context)->y;
   gdk_event_set_device (event, gdk_drag_context_get_device (context));
 
-  gdk_event_put (event);
+  gdk_display_put_event (gdk_window_get_display (window), event);
   gdk_event_free (event);
 }
 
index cb888f0bbc0e4066517c262f65e7e21b7e9f6d61..8f69bee53d3fe254ff47bdd11c472da5c2610a1e 100644 (file)
@@ -124,7 +124,7 @@ selection_buffer_notify (SelectionBuffer *buffer)
       event->selection.time = GDK_CURRENT_TIME;
       event->selection.requestor = g_object_ref (l->data);
 
-      gdk_event_put (event);
+      gdk_display_put_event (gdk_window_get_display (l->data), event);
       gdk_event_free (event);
     }
 }
@@ -563,7 +563,7 @@ gdk_wayland_selection_emit_request (GdkWindow *window,
   event->selection.time = GDK_CURRENT_TIME;
   event->selection.requestor = g_object_ref (window);
 
-  gdk_event_put (event);
+  gdk_display_put_event (gdk_window_get_display (window), event);
   gdk_event_free (event);
 }
 
@@ -1096,7 +1096,7 @@ emit_empty_selection_notify (GdkWindow *requestor,
   event->selection.time = GDK_CURRENT_TIME;
   event->selection.requestor = g_object_ref (requestor);
 
-  gdk_event_put (event);
+  gdk_display_put_event (gdk_window_get_display (requestor), event);
   gdk_event_free (event);
 }
 
@@ -1118,7 +1118,7 @@ emit_selection_clear (GdkDisplay *display,
       event->selection.requestor = g_object_ref (window);
     }
 
-  gdk_event_put (event);
+  gdk_display_put_event (display, event);
   gdk_event_free (event);
 }
 
index 76914f1fddfd377f29dec460cc8eaf566759d2f3..22e55a69d49a8372ec513f6707143d55ed29340f 100644 (file)
@@ -896,7 +896,7 @@ _gdk_device_manager_core_handle_focus (GdkWindow *window,
       if (source_device)
         gdk_event_set_source_device (event, source_device);
 
-      gdk_event_put (event);
+      gdk_display_put_event (gdk_window_get_display (window), event);
       gdk_event_free (event);
     }
 }
index a5d35dfacaa915961c3ea8d6a2436d663b69f22f..a6b76ea85ae41f84e7bc89ee04a2489795d55aa6 100644 (file)
@@ -1256,7 +1256,7 @@ send_client_message_async_cb (Window   window,
       temp_event->dnd.time = GDK_CURRENT_TIME;
       gdk_event_set_device (temp_event, gdk_drag_context_get_device (context));
 
-      gdk_event_put (temp_event);
+      gdk_display_put_event (gdk_drag_context_get_display (context), temp_event);
 
       gdk_event_free (temp_event);
     }
@@ -1309,7 +1309,7 @@ xdnd_send_xevent (GdkX11DragContext *context_x11,
               temp_event->any.window = g_object_ref (window);
 
               if ((*xdnd_filters[i].func) (event_send, temp_event, NULL) == GDK_FILTER_TRANSLATE)
-                gdk_event_put (temp_event);
+                gdk_display_put_event (display, temp_event);
 
               gdk_event_free (temp_event);
 
@@ -2360,7 +2360,7 @@ gdk_x11_drag_context_drag_motion (GdkDragContext *context,
       temp_event->dnd.time = time;
       gdk_event_set_device (temp_event, gdk_drag_context_get_device (context));
 
-      gdk_event_put (temp_event);
+      gdk_display_put_event (gdk_drag_context_get_display (context), temp_event);
       gdk_event_free (temp_event);
     }
   else
@@ -2405,7 +2405,7 @@ gdk_x11_drag_context_drag_motion (GdkDragContext *context,
                 temp_event->dnd.time = time;
                 gdk_event_set_device (temp_event, gdk_drag_context_get_device (context));
 
-                gdk_event_put (temp_event);
+                gdk_display_put_event (gdk_drag_context_get_display (context), temp_event);
                 gdk_event_free (temp_event);
               }
               break;
index cb1f811c9a84be38eccff109acc889ee06b9530d..cbd5c17b037030fce71ebc09d8fa3b4beaf761f4 100644 (file)
@@ -164,7 +164,7 @@ handle_focus_change (GdkEventCrossing *event)
       focus_event->focus_change.in = focus_in;
       gdk_event_set_device (focus_event, gdk_event_get_device ((GdkEvent *) event));
 
-      gdk_event_put (focus_event);
+      gdk_display_put_event (gdk_window_get_display (event->window), focus_event);
       gdk_event_free (focus_event);
     }
 }
@@ -258,7 +258,7 @@ handle_touch_synthetic_crossing (GdkEvent *event)
 
   if (crossing)
     {
-      gdk_event_put (crossing);
+      gdk_display_put_event (gdk_device_get_display (device), crossing);
       gdk_event_free (crossing);
     }
 }